Skip to content

Use constant Charnock parameter in Moon formulation for z0#1

Open
wlanghans wants to merge 2 commits intowolfgang/rebase_sofar-071124_on_mainfrom
wolfgang/use_constant_charnock_in_moon_z0
Open

Use constant Charnock parameter in Moon formulation for z0#1
wlanghans wants to merge 2 commits intowolfgang/rebase_sofar-071124_on_mainfrom
wolfgang/use_constant_charnock_in_moon_z0

Conversation

@wlanghans
Copy link
Copy Markdown

@wlanghans wlanghans commented May 12, 2025

Description

The Moon paper develops a wind-speed based empirical formulation of roughness length for atmosphere-only simulations. The formulation uses a constant value of the Charnock parameter.

That empirical formulation is already mimicking results from wave-atmosphere coupled simulations, and is a fit using that constant value. Using a deviation from Charnock (as a result from our coupling) would alter the results and create a fit different to the Moon paper.

Note:
This branch is a feature on top of this branch, which is our current dev branch rebased onto main

How Has This Been Tested?

Not yet.

Checklist:

Please check all whether they apply or not

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@wlanghans wlanghans changed the base branch from main to wolfgang/rebase_sofar-071124_on_main May 12, 2025 18:44
@wlanghans wlanghans force-pushed the wolfgang/use_constant_charnock_in_moon_z0 branch from 12bbb5e to 7763af4 Compare May 12, 2025 18:53
Copy link
Copy Markdown
Collaborator

@StevePny StevePny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My interpretation of this change is that the moon parameterization should not take a geographically varying charnock parameter, by design, and that changing this does not fit within the scope of the paper.

Is the change from z0 to z0M for the input parameter correct or a typo?

The main from GFDL does not have this:
https://github.com/NOAA-GFDL/SHiELD_physics/blob/d323480c8107be9d7daa15f8782c1bdd75dfb768/gsmphys/sfc_diff_gfdl.f#L417

@wlanghans
Copy link
Copy Markdown
Author

Is the change from z0 to z0M for the input parameter correct or a typo?

Good catch! Thanks.

Copy link
Copy Markdown

@miguelsolanocordoba miguelsolanocordoba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is ok as a reference to check the skill of different SL paramterizations, but we may want to revisit the constants used.

Comment thread gsmphys/sfc_diff_gfdl.f
real(kind=kind_phys) :: ustar_th, z0_adj

real(kind=kind_phys), parameter ::
! & charnock=.014
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wlanghans in the Moon et al. (2007) paper the constants used are charnock=0.0185 (eq. 1 + 8a) and wind_th_moon=12.5 (eq 3-5 + 8a-b). I think this would be good to run by Kun Gao, since he likely changed these from the original formulation based on his own findings.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@miguelsolanocordoba Sure, but can we do this in a separate PR? Here, I'm just trying to make sure that the Charnock number is constant. We can ask Kun in the next meeting; sounds good?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants